Skip to content
This repository was archived by the owner on Jun 15, 2023. It is now read-only.

Move parseSpecification and parseImplementation outside of let rec #515

Merged

Conversation

cknitt
Copy link
Member

@cknitt cknitt commented Jun 1, 2022

No description provided.

@cknitt cknitt force-pushed the make-entrypoint-functions-non-rec branch from 149d0a6 to b4f7418 Compare June 1, 2022 19:19
@IwanKaramazow IwanKaramazow merged commit 7cdefef into rescript-lang:master Jun 1, 2022
@cknitt cknitt deleted the make-entrypoint-functions-non-rec branch June 1, 2022 20:12
(* module signature on the file level *)
let parseSpecification p : Parsetree.signature =
parseRegion p ~grammar:Grammar.Specification ~f:parseSignatureItemRegion
[@@progress (Parser.next, Parser.expect, Parser.checkProgress)]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These annotations go on recursive functions, or the check won't fire.
Moved to parseSignatureItemRegion and parseStructureItemRegion in a PR.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cristianoc Thanks!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, sorry about merging so eagerly

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants